home *** CD-ROM | disk | FTP | other *** search
/ Acorn User: China / Acorn User China CD-ROM (UK) (Disc B) / Acorn User China CD-ROM (UK) (Disc B).bin / BARNET / ARMLINUX / MAIL / 9805 / text0174.txt < prev    next >
Encoding:
Text File  |  1998-06-02  |  2.3 KB  |  86 lines
  1. > I disagree, and believe that it is a good idea to give an error for
  2. > such redeclarations.
  3.  
  4. That defeats the explicit purpose of the feature.
  5.  
  6. > As far as I can tell, you are complaining because it is not possible
  7. > to do:
  8. > 
  9. > typedef union
  10. >   {
  11. >     union wait *__uptr;
  12. >     int *__iptr;
  13. >   } __WAIT_STATUS __attribute__ ((__transparent_union__));
  14. > 
  15. > /* Declaration */
  16. > extern __pid_t __wait __P ((__WAIT_STATUS __stat_loc));
  17. > 
  18. > /* Definition */
  19. > __pid_t
  20. > __wait (int *__stat_loc)
  21. > /* GCC complains about differing types in __wait's signature */
  22. > {
  23. >   /* some expression involving an int *. */
  24. >   return *__stat_loc = 0;
  25. > }
  26.  
  27. I don't care about that per se.  But it *must* be possible to do:
  28.  
  29.     #include <sys/wait.h>
  30.     pid_t wait(int *);
  31.  
  32. and it is highly desireable that it work for a prototype using `union wait *'
  33. too.
  34.  
  35.  
  36. > Now, I suggest that the above sequence of code is in bad style.
  37.  
  38. There is no question about that.  It really doesn't matter to me what
  39. machinations are required in the wait.h decls or in the definitions of the
  40. wait functions.  
  41.  
  42. But redeclarations with prototypes must be allowed; that is the whole point
  43. of this ridiculous kludge.
  44.  
  45. > But, that's only half the story (otherwise, there would be no reason
  46. > to use a transparent union at all).  The macros I define are fine for
  47. > smart implementors of wait, but what about naive users of wait?
  48. > 
  49. > The transparent_union allows them to do:
  50. > 
  51. > {
  52. >   union wait wu;
  53. >   int wi;
  54. >   float wf;
  55. >   wait (&wu);
  56. >   wait (&wi);
  57. >   wait (&wf);
  58. > }
  59. > 
  60. > causing GCC to warn about typecasts only on the last `wait' call, even
  61. > though to the naked eye it appears as if at least one of the other two
  62. > `wait' calls needs a typecast.
  63. > 
  64. > I believe that this behaviour is exactly the intention of the original
  65. > `transparent_union' flag, and that allowing different function
  66. > prototypes is both confusing and unnecessary, and therefore
  67. > undesirable.
  68.  
  69. I happen to know, being the original instigator of the functionality that
  70. became transparent_union.  That is only half of the job originally intended.
  71. The other half is that naive wait users can do:
  72.  
  73.     #include <sys/wait.h>
  74.     ...
  75.     {
  76.       extern pid_t wait(int *);
  77.       int wi;
  78.       float wf;
  79.       wait(&wi);
  80.       wait(&wf);
  81.     }
  82.  
  83. and the only type error should be the last wait call, just as if the
  84. original decl were `pid_t wait(int *);'.
  85.  
  86.